-
Notifications
You must be signed in to change notification settings - Fork 71
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Drop deprecated lxd_container and lxd_container_file resources #377
Drop deprecated lxd_container and lxd_container_file resources #377
Conversation
bc5cbac
to
ae3a4fb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! 🍕
Do you plan to create another PR addressing these? |
Not in the short term so I wouldn't delay the |
@simondeziel can you rebase this? |
af86b19
to
90d8394
Compare
Those have been respectively replaced by lxd_instance and lxd_instance_file. This is more consistent because LXD has been supporting VMs in addition to containers since LXD 3.19. Signed-off-by: Simon Deziel <[email protected]>
…ning/ Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
…anceExpandedDevice/ Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
90d8394
to
43d0bbc
Compare
Added #380 to track the needed deprecations so we don't forget about them. I agree we can handle this post-2.0, and would need to anyway since we still have to have a deprecation cycle. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🍕
Depends on #348
The
lxd_publish_image
andlxd_snapshot
resources still use the legacy namescontainer
andcontainer_name
but I left them for another PR.